Skip to content

fix(release): sign release tags (git tag -s) - #70

Merged
SimplicityGuy merged 2 commits into
mainfrom
fix/release-signed-tag
May 20, 2026
Merged

fix(release): sign release tags (git tag -s)#70
SimplicityGuy merged 2 commits into
mainfrom
fix/release-signed-tag

Conversation

@SimplicityGuy

Copy link
Copy Markdown
Owner

Summary

just release produced an unsigned v1.2.1 tag, whereas v1.2.0 (and prior releases) were PGP-signed. Root cause: the recipe used git tag -a (annotated, unsigned). This switches it to git tag -s so every release tag is signed with the maintainer's configured signing key.

 release version:
     @echo "Creating release v{{version}}..."
-    git tag -a "v{{version}}" -m "Release v{{version}}"
+    git tag -s "v{{version}}" -m "Release v{{version}}"
     git push origin "v{{version}}"

Evidence

  • git cat-file -p v1.2.0 → contains -----BEGIN PGP SIGNATURE-----
  • git cat-file -p v1.2.1 → no signature block
  • Global git signing is configured (it signed v1.2.0), so -s uses the existing key — no new setup required.

Note (separate decision)

The already-published v1.2.1 tag remains unsigned. Re-signing it would require force-updating the pushed tag (and would re-trigger release.yml). Left as-is unless you want me to re-sign/re-push it.

🤖 Generated with Claude Code

SimplicityGuy and others added 2 commits May 19, 2026 20:16
v1.2.0 was PGP-signed but v1.2.1 shipped unsigned because the recipe used
`git tag -a` (annotated, unsigned). Switch to `-s` so `just release`
produces signed tags using the maintainer's configured signing key.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@SimplicityGuy
SimplicityGuy merged commit f58184d into main May 20, 2026
10 checks passed
@SimplicityGuy
SimplicityGuy deleted the fix/release-signed-tag branch May 20, 2026 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant